Skip to content

ci: pin github actions to pull commit SHAs#57

Merged
walkowif merged 1 commit intoinsightsengineering:mainfrom
dzeveloper:pin-used-actions
Mar 31, 2026
Merged

ci: pin github actions to pull commit SHAs#57
walkowif merged 1 commit intoinsightsengineering:mainfrom
dzeveloper:pin-used-actions

Conversation

@dzeveloper
Copy link
Copy Markdown
Contributor

This PR updates the GitHub Action workflows to use full length commit SHAs instead of mutable tags (e.g., @v2).
By pinning to a specific hash, we ensure that the code running in our CI/CD pipeline is exactly what we audited, protecting us from "tag shifting" where a version tag is moved to a different (potentially malicious) commit.

Using version tags like @V3 relies on trust that the maintainer’s account won't be compromised. If an attacker gains access to a dependency we use, they can push a malicious update to that tag, and our pipeline would automatically pull the compromised code.

Reference Incident: aquasecurity/trivy-action#541

@dzeveloper dzeveloper requested a review from a team as a code owner March 26, 2026 14:35
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 26, 2026

✅ All contributors have signed the CLA
Posted by the CLA Assistant Lite bot.

@dzeveloper
Copy link
Copy Markdown
Contributor Author

I have read the CLA Document and I hereby sign the CLA

Copy link
Copy Markdown
Contributor

@walkowif walkowif left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@walkowif walkowif merged commit f3f8cdb into insightsengineering:main Mar 31, 2026
1 of 2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Mar 31, 2026
@walkowif
Copy link
Copy Markdown
Contributor

@dzeveloper Thanks for the PR! It's included in v3.0.3.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants